Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rST Documentation fixes #184

Merged
merged 3 commits into from
Jun 30, 2024
Merged

rST Documentation fixes #184

merged 3 commits into from
Jun 30, 2024

Conversation

scottclowe
Copy link
Contributor

@scottclowe scottclowe commented Jun 29, 2024

Fix the Dataset and collator example code block, the implementation of which was bugged.

  • Need to instantiate the tokenizer
  • Can't pass a positional argument after a keyword argument

Fix rST backtick errors which open with two backticks and close with one backtick.

Fix rST code markup with single backticks to use double backticks instead.


📚 Documentation preview 📚: https://miditok--184.org.readthedocs.build/en/184/

@Natooz
Copy link
Owner

Natooz commented Jun 30, 2024

Thank you so much for theses fixes! 🙌
Merging right away, I'll take care of the lint errors

@Natooz Natooz merged commit ba71555 into Natooz:main Jun 30, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants